Skip to content

Enable BPF target in LLVM backend by default #85630

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Draft
wants to merge 1 commit into
base: main
Choose a base branch
from
Draft

Enable BPF target in LLVM backend by default #85630

wants to merge 1 commit into from
+2 −2

Conversation

@simonjbeaumont
Copy link
Contributor

@simonjbeaumont simonjbeaumont commented Nov 20, 2025 •
edited
Loading

Adding the BPF target to the LLVM backend, allows eBPF programs to be built using the clang that ships with the OSS toolchains.

Should be a low-risk change, which I've tested locally by explicitly specifying the --llvm-targets-to-build flag:

% build/Ninja-ReleaseAssert/llvm-linux-aarch64/bin/clang -print-targets | grep bpf
    bpf        - BPF (host endian)
    bpfeb      - BPF (big endian)
    bpfel      - BPF (little endian)

% touch empty.c

% build/Ninja-ReleaseAssert/llvm-linux-aarch64/bin/clang -target bpf -c empty.c -o empty.o

% file empty.o
empty.o: ELF 64-bit LSB relocatable, eBPF, version 1 (SYSV), not stripped

Looks like a negligible increase in build time vs previous PR toolchain builds:

Job Build script analyzer total LLVM time (sec)
#1584 (this PR)1 2hr 21m 53s 1178.85
#1583 (previous)2 2hr 22m 8s 1171.51
...failed runs... ...failed runs... ...failed runs...
#1578 (previous)3 2hr 22m 40s 1171.47
#1577 (previous)4 2hr 21m 48s 1168.54
#1576 (previous)5 2hr 20m 39s 1174.80

And an increased of size comparison of PR toolchain(s) vs last nightly main, it
looks like it's gone up a bit, but not by this PR:

PR stat -c %s Link
— 1187466342 swift-DEVELOPMENT-SNAPSHOT-2025-08-27-a-ubuntu20.04.tar.gz6
This PR 1212116532 pull-request/85630/1584/ubuntu2204/PR-ubuntu2204.tar.gz7
Previous PR 1213404191 pull-request/85358/1583/ubuntu2204/PR-ubuntu2204.tar.gz8
Previous PR 1213116282 pull-request/79684/1578/ubuntu2204/PR-ubuntu2204.tar.gz9

Footnotes

  1. https://ci.swift.org/job/swift-PR-toolchain-Linux/1584/consoleText ↩

  2. https://ci.swift.org/job/swift-PR-toolchain-Linux/1583/consoleText ↩

  3. https://ci.swift.org/job/swift-PR-toolchain-Linux/1578/consoleText ↩

  4. https://ci.swift.org/job/swift-PR-toolchain-Linux/1577/consoleText ↩

  5. https://ci.swift.org/job/swift-PR-toolchain-Linux/1576/consoleText ↩

  6. https://download.swift.org/development/ubuntu2004/swift-DEVELOPMENT-SNAPSHOT-2025-08-27-a/swift-DEVELOPMENT-SNAPSHOT-2025-08-27-a-ubuntu20.04.tar.gz ↩

  7. https://download.swift.org/tmp/pull-request/85630/1584/ubuntu2204/PR-ubuntu2204.tar.gz ↩

  8. https://download.swift.org/tmp/pull-request/85358/1583/ubuntu2204/PR-ubuntu2204.tar.gz ↩

  9. https://download.swift.org/tmp/pull-request/79684/1578/ubuntu2204/PR-ubuntu2204.tar.gz ↩

@simonjbeaumont
Copy link
Contributor Author

simonjbeaumont commented Nov 20, 2025

@swift-ci Please Build Toolchain Linux Platform

@simonjbeaumont
Copy link
Contributor Author

simonjbeaumont commented Nov 20, 2025

@swift-ci Please Build Toolchain macOS Platform

@simonjbeaumont
Copy link
Contributor Author

simonjbeaumont commented Nov 21, 2025

@swift-ci Please smoke test

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@shahmishal shahmishal Awaiting requested review from shahmishal shahmishal is a code owner

@etcwilde etcwilde Awaiting requested review from etcwilde etcwilde will be requested when the pull request is marked ready for review etcwilde is a code owner

@justice-adams-apple justice-adams-apple Awaiting requested review from justice-adams-apple justice-adams-apple will be requested when the pull request is marked ready for review justice-adams-apple is a code owner

@edymtt edymtt Awaiting requested review from edymtt edymtt will be requested when the pull request is marked ready for review edymtt is a code owner

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@simonjbeaumont